Skip to content

feat: add '包含墨迹' toggle and ink-overlay support for area screenshots#436

Merged
CJKmkp merged 8 commits intoInkCanvasForClass:betafrom
Tayasui-rainnya:beta
Apr 11, 2026
Merged

feat: add '包含墨迹' toggle and ink-overlay support for area screenshots#436
CJKmkp merged 8 commits intoInkCanvasForClass:betafrom
Tayasui-rainnya:beta

Conversation

@Tayasui-rainnya
Copy link
Copy Markdown
Contributor

我嘞个豆啊,浪费额度修那个截屏偏移的bug

这玩意是之前版本带的

目前“包含墨迹”雀食会有一点点偏移,但可以忽略不计了(我1080p,windows缩放100%)

如果说有人测出来墨迹的偏移很大,再来修吧

@CJKmkp CJKmkp self-assigned this Apr 9, 2026
Copy link
Copy Markdown
Member

@CJKmkp CJKmkp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没什么问题

@CJKmkp
Copy link
Copy Markdown
Member

CJKmkp commented Apr 10, 2026

建议迁移net6分支

@CJKmkp CJKmkp requested a review from doudou0720 April 10, 2026 10:05
@doudou0720
Copy link
Copy Markdown
Member

建议迁移net6分支

只有发起pr的人可以改分支

Copy link
Copy Markdown
Member

@doudou0720 doudou0720 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没测实际的,我妈一周现在只让我用2h电脑

@CJKmkp 你自己周末测测罢

@doudou0720 doudou0720 requested a review from CJKmkp April 10, 2026 14:31
Updated XML documentation for ShowScreenshotSelector method to provide clearer details on parameters and return values.
@Tayasui-rainnya
Copy link
Copy Markdown
Contributor Author

建议迁移net6分支

只有发起pr的人可以改分支

awa……我压根没fork那个net6的…………
可以直接提交到net6的分支嘛?

@doudou0720
Copy link
Copy Markdown
Member

@Tayasui-rainnya 可以的,直接改目标分支,
还有你的缩进是不是炸了

@Tayasui-rainnya
Copy link
Copy Markdown
Contributor Author

@Tayasui-rainnya 可以的,直接改目标分支, 还有你的缩进是不是炸了

啊我不到啊
那个docstring我在网页版改的

@Tayasui-rainnya
Copy link
Copy Markdown
Contributor Author

网页版不知道在哪改

@Tayasui-rainnya 可以的,直接改目标分支, 还有你的缩进是不是炸了

@doudou0720
Copy link
Copy Markdown
Member

网页版不知道在哪改

@Tayasui-rainnya 可以的,直接改目标分支, 还有你的缩进是不是炸了

关了重开即可

@doudou0720
Copy link
Copy Markdown
Member

暂时不建议改分支,net6 在不停推commit

@CJKmkp CJKmkp merged commit f776dac into InkCanvasForClass:beta Apr 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants